Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[red-knot] Allow calling bool() with no arguments #13568

Merged
merged 2 commits into from
Sep 30, 2024

Conversation

AlexWaygood
Copy link
Member

Fixes a small oversight in #13538

@AlexWaygood AlexWaygood added the red-knot Multi-file analysis & type inference label Sep 30, 2024
Copy link
Contributor

github-actions bot commented Sep 30, 2024

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@AlexWaygood
Copy link
Member Author

Of course, we'll need to emit an error if more than one argument is passed... but we don't yet do any checking of callable signatures. Baby steps, I guess!

@AlexWaygood AlexWaygood enabled auto-merge (squash) September 30, 2024 13:16
@AlexWaygood AlexWaygood merged commit 5f4b282 into main Sep 30, 2024
17 checks passed
@AlexWaygood AlexWaygood deleted the alex/allow-0-arg-bool branch September 30, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
red-knot Multi-file analysis & type inference
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants